Skip to content

Unify add/edit item form with marks and notes - #36

Merged
amrtgaber merged 1 commit into
mainfrom
feat/unified-item-form
Feb 7, 2026
Merged

Unify add/edit item form with marks and notes#36
amrtgaber merged 1 commit into
mainfrom
feat/unified-item-form

Conversation

@amrtgaber

Copy link
Copy Markdown
Contributor

Summary

  • Remove create-only restriction on marks and notes sections in ItemForm
  • Update EditItemDialog to create marks (with images) and notes after a successful item edit
  • Change uploading message to generic "Saving" since it now covers marks/notes too

Test plan

  • All 15 existing tests pass
  • Build succeeds
  • Manual: edit an item and add marks/notes — verify they appear on the detail page

Remove the create-only restriction on marks and notes sections in
ItemForm, and update EditItemDialog to handle creating marks and notes
after a successful edit.
@netlify

netlify Bot commented Feb 7, 2026

Copy link
Copy Markdown

Deploy Preview for trove-web-production ready!

Name Link
🔨 Latest commit 0ae766d
🔍 Latest deploy log https://app.netlify.com/projects/trove-web-production/deploys/69868a65ef72e00008aa9611
😎 Deploy Preview https://deploy-preview-36--trove-web-production.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@amrtgaber
amrtgaber merged commit f1540b4 into main Feb 7, 2026
6 checks passed
@amrtgaber
amrtgaber deleted the feat/unified-item-form branch February 7, 2026 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant